Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

go mod: script to get a usable mod version from a tag to use in a go.mod #11477

Closed
wants to merge 3 commits into from
Closed

go mod: script to get a usable mod version from a tag to use in a go.mod #11477

wants to merge 3 commits into from

Conversation

obsider
Copy link

@obsider obsider commented Dec 23, 2019

Right now it is impossible to import go.etcd.io/etc v3.X.Y. This script finds the corresponding v0.0.0-...-... from a tag so that it can be included in a project's go.mod.

Fixes #11154 #11336 #11472

@codecov-io
Copy link

codecov-io commented Dec 23, 2019

Codecov Report

Merging #11477 into master will increase coverage by 1.36%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #11477      +/-   ##
==========================================
+ Coverage    64.2%   65.57%   +1.36%     
==========================================
  Files         403      402       -1     
  Lines       38085    36686    -1399     
==========================================
- Hits        24453    24056     -397     
+ Misses      12005    11108     -897     
+ Partials     1627     1522     -105
Impacted Files Coverage Δ
auth/options.go 37.5% <0%> (-55%) ⬇️
client/client.go 48.03% <0%> (-35.95%) ⬇️
proxy/httpproxy/director.go 52.43% <0%> (-27.57%) ⬇️
client/keys.go 68.34% <0%> (-23.12%) ⬇️
pkg/transport/timeout_conn.go 60% <0%> (-20%) ⬇️
etcdserver/api/membership/store.go 60.68% <0%> (-15.67%) ⬇️
etcdserver/api/rafthttp/util.go 74.72% <0%> (-9.23%) ⬇️
proxy/httpproxy/reverse.go 63.02% <0%> (-7.98%) ⬇️
pkg/tlsutil/tlsutil.go 86.2% <0%> (-6.9%) ⬇️
clientv3/leasing/util.go 91.66% <0%> (-6.67%) ⬇️
... and 124 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5adad5e...3d61711. Read the comment docs.

add go.etcd.io/etcd@ to the version for go get

Co-Authored-By: Richard Patel <me@terorie.dev>
@stale
Copy link

stale bot commented Jun 13, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 21 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jun 13, 2020
@stale stale bot closed this Jul 4, 2020
harshavardhana added a commit to harshavardhana/minio that referenced this pull request Dec 3, 2020
Due to botched upstream renames of project repositories
and incomplete migration to go.mod support, our current
dependency version of `go.mod` had bugs i.e it was
using commits from master branch which didn't have
the required fixes present in release-3.4 branches

which leads to some rare bugs

etcd-io/etcd#11477 provides
a workaround for now and we should migrate to this.

release-3.5 eventually claims to fix all of this
properly until then we cannot use /v3 import right now
harshavardhana added a commit to minio/minio that referenced this pull request Dec 3, 2020
Due to botched upstream renames of project repositories
and incomplete migration to go.mod support, our current
dependency version of `go.mod` had bugs i.e it was
using commits from master branch which didn't have
the required fixes present in release-3.4 branches

which leads to some rare bugs

etcd-io/etcd#11477 provides
a workaround for now and we should migrate to this.

release-3.5 eventually claims to fix all of this
properly until then we cannot use /v3 import right now
jsloyer pushed a commit to IBM-Cloud/go-etcd-rules that referenced this pull request May 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

go get error for 3.4.0
3 participants